Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

copy Rt episode in middle of delays episodes #37

Merged
merged 9 commits into from
Apr 2, 2024
Merged

copy Rt episode in middle of delays episodes #37

merged 9 commits into from
Apr 2, 2024

Conversation

avallecam
Copy link
Member

@avallecam avallecam commented Mar 30, 2024 •
edited
Loading

@avallecam avallecam added the upcoming-training To prioritize label Mar 30, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 30, 2024 •
edited
Loading

Thank you!

Thank you for your pull request 😃

🤖 This automated message can help you check the rendered files in your submission for clarity. If you have any questions, please feel free to open an issue in {sandpaper}.

If you have files that automatically render output (e.g. R Markdown), then you should check for the following:

  • 🎯 correct output
  • 🖼️ correct figures
  • ❓ new warnings
  • ‼️ new errors

Rendered Changes

🔍 Inspect the changes: https://github.com/epiverse-trace/tutorials-early/compare/md-outputs..md-outputs-PR-37

The following changes were observed in the rendered markdown documents:

 config.yaml                                        |   1 +
 ...ssibility-rendered-unnamed-chunk-17-1.png (new) | Bin 0 -> 31081 bytes
 ...ssibility-rendered-unnamed-chunk-18-1.png (new) | Bin 0 -> 31019 bytes
 ...ssibility-rendered-unnamed-chunk-21-1.png (new) | Bin 0 -> 84081 bytes
 ...issibility-rendered-unnamed-chunk-5-1.png (new) | Bin 0 -> 5925 bytes
 ...issibility-rendered-unnamed-chunk-8-1.png (new) | Bin 0 -> 10256 bytes
 md5sum.txt                                         |   3 +-
 quantify-transmissibility.md (new)                 | 608 +++++++++++++++++++++
 8 files changed, 611 insertions(+), 1 deletion(-)
What does this mean?

If you have source files that require output and figures to be generated (e.g. R Markdown), then it is important to make sure the generated figures and output are reproducible.

This output provides a way for you to inspect the output in a diff-friendly manner so that it's easy to see the changes that occur due to new software versions or randomisation.

⏱️ Updated at 2024-04-02 21:17:18 +0000

@avallecam avallecam mentioned this pull request Mar 30, 2024
Merged
github-actions bot pushed a commit that referenced this pull request Mar 30, 2024
@actions-user
differences for PR #37
60e6a9d
@avallecam avallecam changed the title copy rt episode in middle of delays episodes copy Rt episode in middle of delays episodes Mar 30, 2024
@Degoot-AM Degoot-AM self-requested a review April 2, 2024 00:08
github-actions bot pushed a commit that referenced this pull request Apr 2, 2024
@actions-user
differences for PR #37
42c4fd3
Degoot-AM
Degoot-AM reviewed Apr 2, 2024
View reviewed changes
episodes/quantify-transmissibility.Rmd Outdated Show resolved Hide resolved
episodes/quantify-transmissibility.Rmd Outdated Show resolved Hide resolved
episodes/quantify-transmissibility.Rmd Show resolved Hide resolved
episodes/quantify-transmissibility.Rmd Outdated Show resolved Hide resolved
episodes/quantify-transmissibility.Rmd Outdated Show resolved Hide resolved
episodes/quantify-transmissibility.Rmd

Let's use `{tidyr}` and `incidence2::incidence()` for this:

```{r, warning = FALSE, message = FALSE}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Be consistent in your usage of pak::func() or just use the func(). It is a personal choice, but I prefer the first option. Because it helps to avoid confusion.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agree for consistency. At least for packages out of the main ones that belong to the main objective of the episode.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added in #42

episodes/quantify-transmissibility.Rmd Outdated Show resolved Hide resolved
episodes/quantify-transmissibility.Rmd Outdated Show resolved Hide resolved
episodes/quantify-transmissibility.Rmd Outdated Show resolved Hide resolved
episodes/quantify-transmissibility.Rmd Outdated Show resolved Hide resolved
@Degoot-AM Degoot-AM self-requested a review April 2, 2024 18:23
@avallecam avallecam mentioned this pull request Apr 2, 2024
Closed
@avallecam avallecam mentioned this pull request Apr 2, 2024
Open
avallecam added a commit to epiverse-trace/tutorials-middle that referenced this pull request Apr 2, 2024
@avallecam
collect feedback + solve lintr checks
dabb0a6 
epiverse-trace/tutorials-early#37
github-actions bot pushed a commit that referenced this pull request Apr 2, 2024
@actions-user
differences for PR #37
064b32f
@avallecam avallecam merged commit 0f6fc2c into main Apr 2, 2024
4 checks passed
@avallecam avallecam deleted the relocate-rt-episode branch April 2, 2024 21:40
avallecam added a commit to epiverse-trace/tutorials-middle that referenced this pull request Apr 2, 2024
@avallecam
collect feedback + solve lintr checks
c5471a6 
epiverse-trace/tutorials-early#37
github-actions bot pushed a commit to epiverse-trace/tutorials-middle that referenced this pull request Apr 2, 2024
@actions-user
markdown source builds
176eed2 
Auto-generated via {sandpaper}
Source  : c5471a6
Branch  : main
Author  : Andree Valle Campos <[email protected]>
Time    : 2024-04-02 21:40:52 +0000
Message : collect feedback + solve lintr checks

epiverse-trace/tutorials-early#37
github-actions bot pushed a commit to epiverse-trace/tutorials-middle that referenced this pull request Apr 2, 2024
@actions-user
site deploy
99ffdaa 
Auto-generated via {sandpaper}
Source  : 176eed2
Branch  : md-outputs
Author  : GitHub Actions <[email protected]>
Time    : 2024-04-02 21:42:23 +0000
Message : markdown source builds

Auto-generated via {sandpaper}
Source  : c5471a6
Branch  : main
Author  : Andree Valle Campos <[email protected]>
Time    : 2024-04-02 21:40:52 +0000
Message : collect feedback + solve lintr checks

epiverse-trace/tutorials-early#37
github-actions bot pushed a commit to epiverse-trace/tutorials-middle that referenced this pull request Apr 9, 2024
@actions-user
site deploy
22c22b8 
Auto-generated via {sandpaper}
Source  : 176eed2
Branch  : md-outputs
Author  : GitHub Actions <[email protected]>
Time    : 2024-04-02 21:42:23 +0000
Message : markdown source builds

Auto-generated via {sandpaper}
Source  : c5471a6
Branch  : main
Author  : Andree Valle Campos <[email protected]>
Time    : 2024-04-02 21:40:52 +0000
Message : collect feedback + solve lintr checks

epiverse-trace/tutorials-early#37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Degoot-AM Degoot-AM Degoot-AM approved these changes

Assignees
No one assigned
Labels
upcoming-training To prioritize
Projects
No open projects
upcoming-training April 11/18/25
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

remove the space before the colon ":" make copy of quantify-transmission episode to tutorials-early
2 participants
@avallecam @Degoot-AM